Skip to content

build: support empty libname flags in configure.py#62477

Merged
nodejs-github-bot merged 1 commit intonodejs:mainfrom
aduh95:from-pkg-config
Mar 30, 2026
Merged

build: support empty libname flags in configure.py#62477
nodejs-github-bot merged 1 commit intonodejs:mainfrom
aduh95:from-pkg-config

Conversation

@aduh95
Copy link
Copy Markdown
Contributor

@aduh95 aduh95 commented Mar 28, 2026

Without this patch, passing e.g. --shared-sqlite-libname= ends up passing -l'' in the lib flags, and there is no way to defer to the pkgconfig ones.

@nodejs-github-bot nodejs-github-bot added build Issues and PRs related to build files or the CI. needs-ci PRs that need a full CI run. labels Mar 28, 2026
@aduh95 aduh95 changed the title build: add a magic string from-pkg-config for shared lib names build: support empty libname flags in configure.py Mar 28, 2026
@nodejs-github-bot

This comment was marked as outdated.

@nodejs-github-bot

This comment was marked as outdated.

@nodejs-github-bot

This comment was marked as outdated.

@nodejs-github-bot

This comment was marked as outdated.

@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

@aduh95 aduh95 added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. commit-queue Add this label to land a pull request using GitHub Actions. labels Mar 30, 2026
@nodejs-github-bot nodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Mar 30, 2026
@nodejs-github-bot nodejs-github-bot merged commit 5913aa9 into nodejs:main Mar 30, 2026
64 checks passed
@nodejs-github-bot
Copy link
Copy Markdown
Collaborator

Landed in 5913aa9

@aduh95 aduh95 deleted the from-pkg-config branch March 30, 2026 13:50
RafaelGSS pushed a commit to RafaelGSS/node that referenced this pull request Mar 30, 2026
PR-URL: nodejs#62477
Reviewed-By: Paolo Insogna <paolo@cowtech.it>
Reviewed-By: Beth Griggs <bethanyngriggs@gmail.com>
Reviewed-By: Chemi Atlow <chemi@atlow.co.il>
Reviewed-By: James M Snell <jasnell@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author ready PRs that have at least one approval, no pending requests for changes, and a CI started. build Issues and PRs related to build files or the CI. needs-ci PRs that need a full CI run.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants